Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Werror flag #751

Closed
wants to merge 358 commits into from
Closed

Werror flag #751

wants to merge 358 commits into from

Conversation

Garra1980
Copy link
Contributor

Please review these guidelines to help with the review process:

  • Have you provided a meaningful PR description?
  • Have you added a test, a reproducer, or a reference to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • If this PR is a work in progress, are you filing the PR as a draft?
  • Have you organized your commits logically and ensured each can be built by itself?

XinyueLunaFan and others added 30 commits April 5, 2023 13:15
It does GEMM on two BF16 matrices and Accumulates in F32 matrix
Source oneapi before configuring IMEX.
Remove redundant SYCL_DIR.
Assumptions:

Github actions is serialized. (Running one action at a time)
CI uses system installed cmake (not from anaconda)

List of changes:

Remove usage of anaconda
Does not cache LLVM build
Local persistent LLVM source repo is shared between cpu and gpu CI. We can do this since CI actions are serialized. cpu and gpu CI actions will pull upstream change an checkout LLVM commit id based on PR.
LLVM/MLIR is always built from scratch
IMEX is built along LLVM as an external project instead of a separate build. This setup cuts LLVM build time since it reduce the number of targets need to be built for LLVM.
Single cmake target "check-imex" is invoked. This target both builds IMEX and runs test.
@Garra1980 Garra1980 closed this May 20, 2024
@Garra1980 Garra1980 deleted the werror_flag branch May 20, 2024 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.